Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI Unknown desktop duration counts #3292

Merged
merged 3 commits into from
Apr 7, 2020
Merged

Conversation

lancewf
Copy link
Contributor

@lancewf lancewf commented Apr 6, 2020

🔩 Description: What code changed, and why?

Adding the initial unknown desktop control to the desktop skeleton page.

⛓️ Related Resources

#3220

👟 How to Build and Test the Change

  1. start_all_services and start the UI
  2. Add ten nodes chef_load_nodes 10
  3. Back date checkin date of the nodes with the below.
    curl -X POST "$ELASTICSEARCH_URL/node-state/_update_by_query?pretty" -H 'Content-Type: application/json' -d'
    {
    "script": {
    "source": "ctx._source[\u0027checkin\u0027] = \u00272020-02-15T02:43:43Z\u0027",
    "lang": "painless"
    }
    }
    '
  4. Open https://a2-dev.test/dashboards/desktop and look for the "Unknown Duration" control with 10 uknown desktop for each duration.

✅ Checklist

📷 Screenshots, if applicable

image

Lance Finfrock added 2 commits April 6, 2020 13:52
Signed-off-by: Lance Finfrock <[email protected]>
Signed-off-by: Lance Finfrock <[email protected]>
@lancewf lancewf marked this pull request as ready for review April 6, 2020 23:19
Copy link
Contributor

@SEAjamieD SEAjamieD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saw the API was merged, but getting an Error Ribbon for Unknown Duration Could not get unknown desktop duration counts errors: Not Found

@lancewf
Copy link
Contributor Author

lancewf commented Apr 7, 2020

@SEAjamieD Thank you for testing it. It might be a little early. We have to wait until the new hab packages are in builder. Sorry about that I jumped the gun a little.

@lancewf
Copy link
Contributor Author

lancewf commented Apr 7, 2020

I am seeing that error now. It looks like it is a problem. I am looking into it now.

@lancewf lancewf changed the title UI Unknown desktop duration counts WIP: UI Unknown desktop duration counts Apr 7, 2020
@lancewf
Copy link
Contributor Author

lancewf commented Apr 7, 2020

It looks like there was a problem pushing up the habitat packages to builder. I am looking into it. This can still be tested by running rebuild components/config-mgmt-service && rebuild components/automate-gateway.

@lancewf lancewf changed the title WIP: UI Unknown desktop duration counts UI Unknown desktop duration counts Apr 7, 2020
Copy link

@vjeffrey vjeffrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

working great after a quick go_update_component automate-gateway and go_update_component config-mgmt-service

Screen Shot 2020-04-07 at 15 29 53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants